[PATCH] [ARM] Fix subtarget feature set truncation when using .cpu directive

Jonathan Roelofs jonathan at codesourcery.com
Wed Feb 4 07:51:44 PST 2015


... other than that, LGTM.


REPOSITORY
  rL LLVM

================
Comment at: lib/Target/ARM/AsmParser/ARMAsmParser.cpp:9186
@@ -9186,2 +9185,3 @@
+  uint64_t FB = ComputeAvailableFeatures(STI.getFeatureBits());
   setAvailableFeatures(FB);
 
----------------
I would change these two lines to:

```
setAvailableFeatures(ComputeAvailableFeatures(STI.getFeatureBits()));
```

http://reviews.llvm.org/D7408

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list