[PATCH] D15075: No error for conflict between inputs\outputs and clobber list

Marina Yatsina via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 27 01:01:14 PST 2016


myatsina added a comment.

In https://reviews.llvm.org/D15075#631207, @vitalybuka wrote:

> These patches break asan tests: http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/528/steps/check-asan%20in%20gcc%20build/logs/stdio


Vitaly ,

This patch is a gcc compatibility issue and it changes clang to output an error in case of illegal inline assembly syntax related to clobber list.

Commit r290540 changed an asan test (asan_asm_test.cc) that used the illegal syntax and fixed it.
The commit removed from the extended inline assembly clobber list registers that also appeared in the  input list.
GCC fails as well on the original inline assembly that appeared in this test, so the fix is correct.
I don't understand why this change has effect the logic of the test - can you help?

Thanks,
Marina


Repository:
  rL LLVM

https://reviews.llvm.org/D15075





More information about the cfe-commits mailing list