Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.

Fix explicit constructor calls and Remove multi-line comments #23162

Merged
merged 11 commits into from
Mar 18, 2019

Conversation

franksinankaya
Copy link

No description provided.

@franksinankaya
Copy link
Author

@jkotas @janvorli @am11

@franksinankaya franksinankaya changed the title Fix implicit constructor calls and Remove multi-line comments Fix explicit constructor calls and Remove multi-line comments Mar 10, 2019
@jkotas
Copy link
Member

jkotas commented Mar 10, 2019

cc @dotnet/jit-contrib

@franksinankaya franksinankaya force-pushed the gcc_cleanup_10 branch 2 times, most recently from 49febc3 to fb813be Compare March 10, 2019 04:28
src/jit/morph.cpp Outdated Show resolved Hide resolved
@franksinankaya franksinankaya force-pushed the gcc_cleanup_10 branch 5 times, most recently from 7bf6b4a to f668cce Compare March 11, 2019 05:16
src/jit/morph.cpp Outdated Show resolved Hide resolved
src/jit/unwind.h Outdated Show resolved Hide resolved
@franksinankaya franksinankaya force-pushed the gcc_cleanup_10 branch 12 times, most recently from 03ed09c to f3b1eda Compare March 14, 2019 01:25
src/jit/emit.h Show resolved Hide resolved
src/jit/unwindarm.cpp Outdated Show resolved Hide resolved
src/jit/unwindarm.cpp Outdated Show resolved Hide resolved
src/jit/unwindarm.cpp Outdated Show resolved Hide resolved
@franksinankaya
Copy link
Author

unrelated compilation error on arm:

19:59:53 [ 26%] Built target FromNativePaths_lib
19:59:53
/mnt/j/workspace/dotnet_coreclr/master/arm_cross_checked_ubuntu_innerloop_prtest/tests/src/Common/hostpolicymock/HostpolicyMock.cpp:35:14: error: calling convention '__cdecl' ignored for this target [-Werror,-Wignored-attributes]
19:59:53 typedef void(__cdecl *Callback_corehost_resolve_component_dependencies)(const char_t *component_main_assembly_path);
19:59:53

let me rebase one more time.

@franksinankaya
Copy link
Author

Is this good to go in?

@franksinankaya franksinankaya force-pushed the gcc_cleanup_10 branch 2 times, most recently from f0ef607 to ef6e009 Compare March 18, 2019 02:00
@franksinankaya
Copy link
Author

@dotnet-bot retest Windows_NT x64 Checked CoreFX Tests

@franksinankaya
Copy link
Author

@BruceForstall : can you review the newer version?

@BruceForstall BruceForstall merged commit 6100a9f into dotnet:master Mar 18, 2019
@franksinankaya franksinankaya deleted the gcc_cleanup_10 branch March 18, 2019 18:30
sandreenko pushed a commit to sandreenko/coreclr that referenced this pull request Mar 18, 2019
…#23162)

* fix implicit constructor call

* extern c

format patch

* muti-line

* Remove direct constructor call

* Conversion

* Need paranthesis

* Return value on resize

* declspec(Thread)

* Ignore warnings for GCC

* Formatting issues

* Move cast to constant
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
…/coreclr#23162)

* fix implicit constructor call

* extern c

format patch

* muti-line

* Remove direct constructor call

* Conversion

* Need paranthesis

* Return value on resize

* declspec(Thread)

* Ignore warnings for GCC

* Formatting issues

* Move cast to constant


Commit migrated from dotnet/coreclr@6100a9f
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants