[PATCH] D46541: [CodeGen] Improve diagnostics related to target attributes

Gabor Buella via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 21 05:53:39 PDT 2018


GBuella updated this revision to Diff 147775.
GBuella added a comment.

Fixed a horrible bug in the patch. Adding a ref to temporary string is not a wise thing to do, so I had to remove this line:

  ReqFeatures.push_back(F.substr(1));

Now the ReqFeatures vector can also refer to strings starting with '+'

Also, added a few tests and comments.


https://reviews.llvm.org/D46541

Files:
  lib/CodeGen/CodeGenFunction.cpp
  lib/CodeGen/CodeGenModule.cpp
  lib/CodeGen/CodeGenModule.h
  test/CodeGen/target-features-error-2.c
  test/CodeGen/target-features-error.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D46541.147775.patch
Type: text/x-patch
Size: 8529 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180521/54fb5975/attachment.bin>


More information about the cfe-commits mailing list